Skip to content

Comments

Fix Radar to display agent-jwks identities#47

Closed
runnerelectrode wants to merge 1 commit intoOpenBotAuth:mainfrom
runnerelectrode:fix/radar-agent-jwks-display
Closed

Fix Radar to display agent-jwks identities#47
runnerelectrode wants to merge 1 commit intoOpenBotAuth:mainfrom
runnerelectrode:fix/radar-agent-jwks-display

Conversation

@runnerelectrode
Copy link
Contributor

Summary

  • formatAgentName in Radar.tsx only parsed /jwks/username.json URLs (user JWKS format) but not /agent-jwks/{uuid} URLs (agent JWKS format from API registration via POST /agents)
  • Agents registered via the API (e.g. using clawauth) had their identity displayed as api.openbotauth.org instead of a meaningful identifier
  • Added a second regex to match /agent-jwks/{id} and display a truncated agent ID (first 8 chars + ellipsis)

Before: Agent shows as api.openbotauth.org
After: Agent shows as 9c7f611e…

Longer-term suggestion

Ideally the Radar API (/telemetry/top/agents) should return the client_name from the agent record so the display shows the actual agent name (e.g. "TARS_OBA") instead of parsing the URL. The TopAgent interface already has client_name — it just needs to be populated by the backend.

Test plan

  • Verify /jwks/username.json URLs still display as username
  • Verify /agent-jwks/{uuid} URLs now display as {first-8-chars}… instead of api.openbotauth.org
  • Verify known agents (chatgpt.com, etc.) still get friendly names

🤖 Generated with Claude Code

The formatAgentName function only parsed /jwks/username.json URLs
(user JWKS format) but not /agent-jwks/{uuid} URLs (agent JWKS
format from API registration). Agents registered via the API had
their identity displayed as "api.openbotauth.org" instead of a
meaningful identifier.

Add a second regex to match /agent-jwks/{id} and display a
truncated agent ID (first 8 chars).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@runnerelectrode
Copy link
Contributor Author

Closing — the /agent-jwks/{uuid} endpoint is being rolled back per #6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant